Skip to content

Make sudo installation optional and add new cases for different OS dist. #136

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Sep 8, 2025
Merged

Make sudo installation optional and add new cases for different OS dist. #136

merged 2 commits into from
Sep 8, 2025
+112 −17

Conversation

AlirezaShamsoshoara
Copy link
Member

This PR addresses makes the installation optional for sudo user:

  • The default for the installation would be on CONDA channel unless user specifies with an input argument (--use-sudo).
  • Also, the installation now has different cases for different OS distributions that can be added to the script.
  • The README file is updated accordingly.
  • The script is tested on Nvidia DevGPU & CoreWeave node for three and eight GPUs.

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Sep 8, 2025
@allenwang28
Copy link
Contributor

Note - error is unrelated to this PR / due to TorchStore which @LucasLLC will look into later

@allenwang28 allenwang28 merged commit d74c149 into meta-pytorch:main Sep 8, 2025
2 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@allenwang28 allenwang28 allenwang28 approved these changes

Assignees

No one assigned

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@AlirezaShamsoshoara @allenwang28